Skip to content

Conversation

marcoieni
Copy link
Member

@marcoieni marcoieni commented May 30, 2025

By using a free runner, this PR job now takes 70 minutes instead of 60. I think it's fine because soon #136942 should reduce how long it takes to run this job anyway.

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels May 30, 2025
@marcoieni
Copy link
Member Author

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 7ad2c36 with merge 83c1482

rust-bors bot added a commit that referenced this pull request May 30, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job

<!-- homu-ignore:start -->
<!--
If this PR is related to an unstable feature or an otherwise tracked effort,
please link to the relevant tracking issue here. If you don't know of a related
tracking issue or there are none, feel free to ignore this.

This PR will get automatically assigned to a reviewer. In case you would like
a specific user to review your work, you can assign it to them by using

    r? <reviewer name>
-->
r? `@ghost`
<!-- homu-ignore:end -->
@marcoieni
Copy link
Member Author

I made a mistake in running bors try, since this is a pr job 😅

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@marcoieni
Copy link
Member Author

r? @Kobzol

@marcoieni marcoieni marked this pull request as ready for review June 2, 2025 07:52
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 2, 2025
@Kobzol
Copy link
Member

Kobzol commented Jun 2, 2025

We need to reduce the costs here, so even taking the 60 -> 70 minute hit isn't the end of the world, although it is slightly annoying.

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 2, 2025

📌 Commit 7ad2c36 has been approved by Kobzol

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 2, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 2, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 2, 2025
ci: use free runner for aarch64-gnu-llvm-19-1 PR job
bors added a commit that referenced this pull request Jun 2, 2025
Rollup of 5 pull requests

Successful merges:

 - #141767 (ci: use free runner for aarch64-gnu-llvm-19-1 PR job)
 - #141858 (Fix typo in `StructuralPartialEq` docs)
 - #141865 (Optionally don't steal the THIR)
 - #141874 (add f16_epsilon and f128_epsilon diagnostic items)
 - #141904 (test-float-parse: apply `cfg(not(bootstrap))`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 2cb5b0d into rust-lang:master Jun 2, 2025
9 checks passed
@rustbot rustbot added this to the 1.89.0 milestone Jun 2, 2025
rust-timer added a commit that referenced this pull request Jun 2, 2025
Rollup merge of #141767 - marcoieni:free-pr-job, r=Kobzol

ci: use free runner for aarch64-gnu-llvm-19-1 PR job
jieyouxu pushed a commit to jieyouxu/rustc-dev-guide that referenced this pull request Jun 3, 2025
Rollup of 5 pull requests

Successful merges:

 - rust-lang/rust#141767 (ci: use free runner for aarch64-gnu-llvm-19-1 PR job)
 - rust-lang/rust#141858 (Fix typo in `StructuralPartialEq` docs)
 - rust-lang/rust#141865 (Optionally don't steal the THIR)
 - rust-lang/rust#141874 (add f16_epsilon and f128_epsilon diagnostic items)
 - rust-lang/rust#141904 (test-float-parse: apply `cfg(not(bootstrap))`)

r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants